Skip to content

Encode form name#2923

Merged
labkey-matthewb merged 8 commits intodevelopfrom
fb_encodeFormName
Mar 30, 2026
Merged

Encode form name#2923
labkey-matthewb merged 8 commits intodevelopfrom
fb_encodeFormName

Conversation

@labkey-matthewb
Copy link
Copy Markdown
Contributor

@labkey-matthewb labkey-matthewb commented Mar 25, 2026

@labkey-nicka labkey-nicka removed their request for review March 30, 2026 16:16
labkey-matthewb added a commit to LabKey/platform that referenced this pull request Mar 30, 2026
#### Rationale
Whereas browsers seem to have trouble property formatting
multipart/form-data POSTs, we will encode form names that include these
three characters: `% " \`

For backward compatibility we will identify encoded field names by
prepending `%_`.

`quf_name` -> `quf_name`
`quf_why"?` -> `%_quf_why%22%3F`

#### Related Pull Requests
- LabKey/labkey-api-js#211
- LabKey/labkey-ui-components#1961
- LabKey/labkey-ui-premium#916
- LabKey/limsModules#2061
- #7515
- LabKey/testAutomation#2923

#### Changes
- <!-- list of descriptions of changes that are worth noting (replace
this comment) -->

---------

Co-authored-by: labkey-nicka <nickk@labkey.com>
@labkey-matthewb labkey-matthewb merged commit 8d36e73 into develop Mar 30, 2026
5 checks passed
@labkey-matthewb labkey-matthewb deleted the fb_encodeFormName branch March 30, 2026 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants